Skip to content

allow float as weight in afni's Allineate #2204

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 4, 2017

Conversation

salma1601
Copy link
Contributor

@salma1601 salma1601 commented Oct 3, 2017

AFNI's 3dAllineate accepts either float or dataset as weight

@salma1601 salma1601 changed the title allow float as weight allow float as weight in afni's Allineate Oct 3, 2017
@codecov-io
Copy link

codecov-io commented Oct 3, 2017

Codecov Report

Merging #2204 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2204      +/-   ##
==========================================
+ Coverage   72.27%   72.28%   +<.01%     
==========================================
  Files        1175     1175              
  Lines       58778    58779       +1     
  Branches     8460     8460              
==========================================
+ Hits        42483    42486       +3     
+ Misses      14933    14931       -2     
  Partials     1362     1362
Flag Coverage Δ
#smoketests 72.28% <100%> (ø) ⬆️
#unittests 69.95% <100%> (ø) ⬆️
Impacted Files Coverage Δ
...ipype/interfaces/afni/tests/test_auto_Allineate.py 85.71% <ø> (ø) ⬆️
nipype/interfaces/afni/preprocess.py 81.29% <100%> (+0.02%) ⬆️
nipype/interfaces/base.py 83.48% <0%> (+0.18%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a988d6...40b66be. Read the comment docs.

Copy link
Contributor

@oesteban oesteban left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I think a deprecation would be better than a complete removal of the old option. Sorry for the overhead and thanks a lot for the contribution.

@@ -353,12 +353,13 @@ class AllineateInputSpec(AFNICommandInputSpec):
argstr='-nomask',
desc='Don\'t compute the autoweight/mask; if -weight is not '
'also used, then every voxel will be counted equally.')
weight_file = File(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFNI Allineate has been there for a long while. Would it make sense to leave the weight_file input here with the deprecated meta set?

    weight_file = File(
        argstr='-weight %s',
        exists=True,
        deprecated='1.0.0', new_name='weight',
        desc='Set the weighting for each voxel in the base dataset; '
             'larger weights mean that voxel count more in the cost function. '
             'Must be defined on the same grid as the base dataset')

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it make sense to leave the weight_file input here with the deprecated meta set?

Ok, sorry.

@oesteban oesteban merged commit 3fde2fb into nipy:master Oct 4, 2017
@satra satra added this to the 0.14.0 milestone Oct 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants